-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: add urcrypt
#90
Conversation
c5d3e99
to
6b7e21b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marker review to make sure we audit the licenses of everything going into this codebase. @matthew-levan please reply to this review comment with a checklist of vendored libraries and a list of their licenses.
b601e9e
to
f1c0eb1
Compare
f1c0eb1
to
ef382a8
Compare
61ccc2e
to
d893931
Compare
d893931
to
589899b
Compare
296f437
to
58af42e
Compare
e2331d3
to
4ea4624
Compare
815c845
to
8a07ad5
Compare
f5b2a67
to
40e04e8
Compare
@@ -14,5 +14,6 @@ pkgs.mkShell { | |||
]) | |||
cargo-watch | |||
gdb | |||
secp256k1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
urcrypt
should be built in the Nix environment. We should not require global installs of any package for developers.
I'll leave this open for now but it's likely to close in favor of #123 shortly. |
Adds
urcrypt-sys
, a crate that provides Rust bindings tourcrypt
, which is now required as a system dependency for building Ares. Also adds dependency installation commands in the CI script.